-
-
Notifications
You must be signed in to change notification settings - Fork 4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Scheduling plugin not updating responding attendee status #28094
Conversation
Signed-off-by: Anna Larch <anna@nextcloud.com>
3879be5
to
29922f7
Compare
Shall we backport this? |
/backport to stable22 |
/backport to stable21 |
/backport to stable20 |
Drone failure is unrelated |
I'm not able to reproduce the initial issue on master anymore. On master and this branch the attendance status is updated properly and shows the right state 😕 |
My bad 🙈 I looked at the wrong point. For master and this pull request the attendance status for the organizer is updated properly. The purpose of this pull request is to fix an issue with updating the attendance status for the attendee itself. To reproduce create a event and invite Alice and Bob. Wait for Bob's invitation email and accept the invite. The attendance status for the organizer and Alice is updated (Bob accepted the invite). In Bob's calendar event the attendance status is still pending. A possible explanation for this case is the way we handle event invitations. If a user accept or decline an invitation a iTip message is generated. server/apps/dav/lib/Controller/InvitationResponseController.php Lines 200 to 201 in 215aef3
The recipient is used to select the right event from the calendar. When Alice accept the invite an iTip message is generated with sender = alice@example.org and recipient = organizer@example.org. The event for recipient organizer@example.org is selected and the attendance status for alice@example.org updated. Then for every other attendee another iTip message is generated to also update their event. Sabre assume that we already updated Alice event and there is no need to sent a iTip message to Alice.
I'm not sure if that's a valid iTip message. But when sender = recipient the calendar event for the attendee is updated and a iTip message for the organizer and other attendees generated and properly updated. Just adding my findings here to document it for later. I guess it's fine to merge this pull request and take out the ignore check. |
This has been discussed an the actual RFC compliant way will be to update the recipient in the iTip Message. Thanks @kesselb for figuring this out! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I tested it and the status was updated correctly for both the organizer and attendee. I didn't look at the code though.
Fixes nextcloud/calendar#2861